Adminhtml: improved info for order sent status #4623
Conversation
github-actions
bot
added
Template : admin
|
This PR needs changes before merging. |
|
For change wordings please make suggestions via review (select lines & ctrl+h). |
app/design/adminhtml/default/default/template/sales/order/view/info.phtml
Outdated
Show resolved
Hide resolved
|
Regarding the positioning of the copy icon, the issue is still visible in the legacy theme.
|
|
There was a problem hiding this comment.
Pull Request Overview
This pull request improves the visibility and presentation of the order confirmation email status in the admin interface by moving it from the order header to a dedicated row in the order information table and updating the visual styling of copy icons.
- Moves email sent status from order header to dedicated table row with clearer labels
- Updates CSS styling to float copy icons to the right for better visual alignment
- Refactors code for better readability by removing inline conditional logic
Reviewed Changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| app/design/adminhtml/default/default/template/sales/order/view/info.phtml | Relocates email status from header to table row, adds new translation strings, improves code formatting |
| app/locale/en_US/Mage_Sales.csv | Adds translation strings for "Order Confirmation Email", "Sent", and "Not sent" |
| skin/adminhtml/default/default/boxes.css | Updates copy icon styling with better margins and right float positioning |
| skin/adminhtml/default/openmage/override.css | Adds right float positioning for copy icons |
| skin/adminhtml/default/openmage/scss/override.scss | Adds right float positioning for copy icons in SCSS source |
app/design/adminhtml/default/default/template/sales/order/view/info.phtml
Outdated
Show resolved
Hide resolved
…/info.phtml Co-authored-by: Copilot <[email protected]>
app/design/adminhtml/default/default/template/sales/order/view/info.phtml
Outdated
Show resolved
Hide resolved
|
Please retry analysis of this Pull-Request directly on SonarQube Cloud |
Description (*)
See #4620 (comment)
Fixed Issues (if relevant)